Skip to content

Reuse UnsafeIntrinsicCompiler inside TrampolineCompiler#13601

Merged
fitzgen merged 1 commit into
bytecodealliance:mainfrom
fitzgen:dedupe-intrinsics-compilation
Jun 10, 2026
Merged

Reuse UnsafeIntrinsicCompiler inside TrampolineCompiler#13601
fitzgen merged 1 commit into
bytecodealliance:mainfrom
fitzgen:dedupe-intrinsics-compilation

Conversation

@fitzgen

@fitzgen fitzgen commented Jun 9, 2026

Copy link
Copy Markdown
Member

Rather than duplicating all the compilation of unsafe intrinsics.

Follow up to #13597; depends on that PR.

@fitzgen fitzgen requested review from a team as code owners June 9, 2026 18:28
@fitzgen fitzgen requested review from alexcrichton and removed request for a team June 9, 2026 18:28

@cfallin cfallin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM on last commit (thanks!).

@github-actions github-actions Bot added the wasmtime:api Related to the API of the `wasmtime` crate itself label Jun 9, 2026
Rather than duplicating all the compilation of unsafe intrinsics.
@fitzgen fitzgen force-pushed the dedupe-intrinsics-compilation branch from c3dc45b to c38092b Compare June 10, 2026 19:06
@fitzgen fitzgen enabled auto-merge June 10, 2026 19:06
@fitzgen fitzgen added this pull request to the merge queue Jun 10, 2026
Merged via the queue into bytecodealliance:main with commit 3e787e6 Jun 10, 2026
51 checks passed
@fitzgen fitzgen deleted the dedupe-intrinsics-compilation branch June 10, 2026 19:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

wasmtime:api Related to the API of the `wasmtime` crate itself

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants